Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Score Postprocessing in QA Task for English and French Versions #613

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

benhachy
Copy link

@benhachy benhachy commented Sep 14, 2023

Fixed scores postprocessing for the QA task specifically when using torch.triu, which sometimes gives the same start and end indices if the diagonal parameter is not set to 1 , and similarly the np.triu with the k parameter, which may give same start and end indices if k is not set to 1.

  • Both English and French version are fixed in this Pull Request.
  • Both Torch and Tensorflow code is fixed in this Pull Request.

…orch.triu which sometimes gives the same start and end indices if the diagonal parameter is not set to 1, and similarly the np.triu with the k parameter.
@benhachy
Copy link
Author

benhachy commented Sep 14, 2023

It is quite weird why the Build didn't succeed as I didn't change the code of the Spanish version of the course, where it seems to be a build error 🤷 @lewtun

Actually it seems that the Build Error appeared in other Pull Requests as #611 and #609 ...

@osanseviero
Copy link
Contributor

Sorry for the issue, the error is unrelated to your PR!

@benhachy
Copy link
Author

Yeah I thought so !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants